home *** CD-ROM | disk | FTP | other *** search
/ Atari Mega Archive 1 / Atari Mega Archive - Volume 1.iso / lists / mint / l_0799 / 458 < prev    next >
Encoding:
Internet Message Format  |  1994-08-27  |  5.8 KB
  1. Subject: Re: Bug report. MiNT 1.07.
  2. Date: Sat, 28 Aug 93 2:01:20 CES
  3. From: Juergen Lock <nox@jelal.north.de>
  4. In-Reply-To: <428.9308261557@earth.ox.ac.uk>; from "Stephen Usher" at Aug 26, 93 4:57 pm
  5. Message-Id: <9308280001.AA00720@jelal.north.de>
  6.  
  7. Stephen Usher writes:
  8.  
  9. > Just a quick bug report, I've not time for anything more.
  10. > 
  11. > Machine:- TT030, 8MB RAM (4+4), standard TT Hard drive + ICD adaptor and
  12. > 85MB seagate drive on ACSI bus.
  13. > 
  14. > (1) When opening serial ports modem2, serial1 and serial2, modem1 is opened.
  15.  
  16.  hmm haven't seen that one...  actually my uucico (taylor 1.03) just
  17. received this mail over modem2, without any problems.  this is MiNT 1.09...
  18. > 
  19. > I noticed this the other day when I tried to use my own port of Kermit
  20. > 5A(188).
  21.  
  22.  btw is it available for ftp/whatever? :)
  23.  
  24. > (2) When MiNT is rebooted after successfully exiting, system soft-crashes.
  25.  
  26.  if you mean exit MiNT -> desktop -> run MiNT again (i.e. without reset)
  27. i think i just found the reason:  close on /dev/fasttext didn't fix
  28. the cursor address to point into the original screen memory again...
  29. and while i was at it i also made open work more than once and fixed a
  30. `strangely' flashing cursor when it was on before the open.  (btw the
  31. scr_usecnt is necessary because /dev/console and fasttext use the same
  32. tty struct...)
  33.  
  34.  and after that i have a few patches for init, login, passwd etc :-)
  35.  
  36. Index: dosmem.c
  37. --- dosmem.c_    Tue Aug 17 18:36:24 1993
  38. +++ dosmem.c    Mon Aug 23 16:45:16 1993
  39. @@ -1294,7 +1294,10 @@
  40.      sigmask = curproc->sigmask;
  41.      curproc->sigmask = ~((unsigned long)1 << SIGKILL);
  42.  
  43. +    { short sr = spl7();
  44.      add_q(READY_Q, p);        /* put it on the ready queue */
  45. +    spl(sr);
  46. +    }
  47.      sleep(WAIT_Q, (long)p);            /* while we wait for it */
  48.      TRACE(("do_vfork: parent waking up"));
  49.  
  50.  (not sure if this was necessary, just in case...)
  51.  
  52. Index: fasttext.c
  53. --- fasttext.c_    Tue Aug 17 11:11:16 1993
  54. +++ fasttext.c    Wed Aug 25 17:06:24 1993
  55. @@ -20,6 +20,7 @@
  56.  #define CONDEV    (2)
  57.  
  58.  static SCREEN *current;
  59. +static short scr_usecnt;
  60.  
  61.  static void paint P_((SCREEN *, int, char *)),
  62.       paint8c P_((SCREEN *, int, char *)),
  63. @@ -171,11 +172,13 @@
  64.          if (hardbase == 0) {
  65.              ALERT("Insufficient memory for hardware scrolling!");
  66.          } else {
  67. +            v->curstimer = 0x7f;
  68.              quickmove(hardbase, base, scrnsize);
  69.              v->cursaddr = v->cursaddr + (hardbase - base);
  70.              oldbase = base;
  71.              base = hardbase;
  72.              Setscreen(hardbase, hardbase, -1);
  73. +            v->curstimer = v->period;
  74.          }
  75.      }
  76.      hardline = 0;
  77. @@ -214,7 +217,9 @@
  78.          v->flags &= ~CURS_FSTATE;
  79.  
  80.      /* now turn the cursor on the way we like it */
  81. +        v->curstimer = v->period;
  82.          v->hidecnt = 0;
  83. +        v->flags |= CURS_ON;
  84.          curs_on(v);
  85.      } else {
  86.          (void)Cursconf(0,0);
  87. @@ -1201,8 +1206,13 @@
  88.  
  89.      if (!current) {
  90.          init();
  91. -    } else
  92. +    }
  93. +#if 1
  94. +    ++scr_usecnt;
  95. +#else
  96. +    else
  97.          return EACCDN;        /* screen in use */
  98. +#endif
  99.  
  100.      f->flags |= O_TTY;
  101.      return 0;
  102. @@ -1213,13 +1223,21 @@
  103.      FILEPTR *f;
  104.      int pid;
  105.  {
  106. +    SCREEN *v = current;
  107.      UNUSED(pid);
  108.  
  109. -    if (f->links <= 0) {
  110. +#if 1
  111. +    if (v && f->links <= 0 && !--scr_usecnt) {
  112. +#else
  113. +    if (v && f->links <= 0) {
  114. +#endif
  115.          if (hardbase) {
  116. +            v->curstimer = 0x7f;
  117. +            v->cursaddr = v->cursaddr + (oldbase - base);
  118.              quickmove(oldbase, base, scrnsize);
  119.              base = oldbase;
  120.              Setscreen(oldbase, oldbase, -1);
  121. +            v->curstimer = v->period;
  122.          }
  123.          current = 0;
  124.      }
  125.  
  126. -------end mint patches----
  127.  
  128.  this makes the tty always get the right pgrp:
  129.  
  130. Index: init/init.c
  131. --- /i/tmp/init/init.c    Mon Aug  2 23:58:08 1993
  132. +++ /f/msrc/ini11/init/init.c    Tue Aug 24 13:14:56 1993
  133. @@ -44,6 +44,7 @@
  134.  #include <sys/resource.h>
  135.  
  136.  #ifdef MINT
  137. +#include <ioctl.h>
  138.  #include <sys/dir.h>
  139.  #include <mintbind.h>
  140.  
  141. @@ -493,12 +494,17 @@
  142.  
  143.      if ((pid = vfork()) == 0)
  144.      {
  145. -        setpgrp(getpid(), getpid());
  146. +        long pgrp = setpgrp(getpid(), getpid());
  147.  
  148.          tty = open(ttyname, O_RDWR);
  149.  
  150.  #ifdef MINT
  151. +        if (tty == -1 || pgrp < 0) {
  152. +            perror(ttyname);
  153. +            _exit(0);
  154. +        }
  155.          dup2(tty, -1);        /* set controlling terminal */
  156. +        ioctl (-1, TIOCSPGRP, &pgrp);
  157.  #endif
  158.          dup2(tty, 0);
  159.          dup2(tty, 1);
  160.  
  161.  i think the next ones are also the reason why login crashed for some
  162. people...
  163.  
  164. Index: login/login.c
  165. --- /i/tmp/login/login.c    Wed Aug  4 01:46:44 1993
  166. +++ /f/msrc/ini11/login/login.c    Tue Aug 24 11:34:44 1993
  167. @@ -384,7 +384,7 @@
  168.  
  169.      if (i < argc)
  170.      {
  171. -        if ((j = strlen(argv[i])) > 8)
  172. +        if ((j = strlen(argv[i])+1) > 8)
  173.              j = 8;
  174.          strncpy(name, argv[i], j);
  175.          name[8] = '\0';
  176. Index: login/passwd.c
  177. --- /i/tmp/login/passwd.c    Mon Aug  2 23:58:08 1993
  178. +++ /f/msrc/ini11/login/passwd.c    Tue Aug 24 11:28:32 1993
  179. @@ -202,7 +202,7 @@
  180.  
  181.      if (i < argc)
  182.      {
  183. -        if ((j = strlen(argv[i])) > 8)
  184. +        if ((j = strlen(argv[i])+1) > 8)
  185.              j = 8;
  186.          strncpy(name, argv[i], j);
  187.          name[8] = '\0';
  188. Index: login/su.c
  189. --- /i/tmp/login/su.c    Wed Aug  4 01:46:18 1993
  190. +++ /f/msrc/ini11/login/su.c    Tue Aug 24 11:34:52 1993
  191. @@ -298,7 +298,7 @@
  192.  
  193.      if (i < argc)
  194.      {
  195. -        if ((j = strlen(argv[i])) > 8)
  196. +        if ((j = strlen(argv[i])+1) > 8)
  197.              j = 8;
  198.          strncpy(name, argv[i], j);
  199.          name[8] = '\0';
  200.  
  201.  and getty restores tty on some more signals:
  202.  
  203. Index: getty/getty.c
  204. --- getty/getty.c_    Mon Aug  2 23:58:08 1993
  205. +++ getty/getty.c    Thu Aug 26 22:10:02 1993
  206. @@ -561,8 +561,12 @@
  207.      char *s = "Connection timed out.\r\n";
  208.  
  209.      ioctl(0, TIOCSETP, &save_sgttyb);
  210. -    write(0, s, strlen(s));
  211. -    exit(1);
  212. +    if (sig == SIGALRM) {
  213. +        write(0, s, strlen(s));
  214. +        exit(1);
  215. +    }
  216. +    signal (sig, SIG_DFL);
  217. +    kill (getpid(), sig);
  218.  }
  219.  
  220.  int
  221. @@ -582,6 +586,10 @@
  222.  
  223.      ioctl(0, TIOCGETP, &save_sgttyb);
  224.      signal(SIGALRM, do_alarm);
  225. +    signal(SIGHUP, do_alarm);
  226. +    signal(SIGINT, do_alarm);
  227. +    signal(SIGQUIT, do_alarm);
  228. +    signal(SIGTERM, do_alarm);
  229.  
  230.  #ifdef DEBUG
  231.      if ((dbgfp = fopen(DEBUGFILE, "w+")) == NULL)
  232.  
  233.  hope this helps some...  cheers,
  234.     Juergen
  235. -- 
  236. J"urgen Lock / nox@jelal.north.de / UUCP: ..!uunet!unido!uniol!jelal!nox
  237.                                 ...ohne Gewehr
  238. PGP public key fingerprint =  8A 18 58 54 03 7B FC 12  1F 8B 63 C7 19 27 CF DA 
  239.